gtk: Stop using modifier intents

Reviewing the existing settings, the only backend with
some differences in the modifier intent settings is OS X,
and we would rather have that implemented by interpreting
the existing modifiers in the appropriate way.

                X11      Wayland  Win32    OS X

primary         ctrl     ctrl     ctrl     mod2
mnemonic        alt      alt      alt      alt
context menu    -        -        -        ctrl
extend sel      shift    shift    shift    shift
modify sel      ctrl     ctrl     ctrl     mod2
no text         alt|ctrl alt|ctrl alt|ctrl mod2|ctrl
shift group     varies   -        -        alt

GTK now uses the following modifiers:

primary         ctrl
mnemonic        alt
extend sel      shift
modify sel      ctrl
no text         alt|ctrl

The context menu and shift group intents were not used
in GTK at all.

Update tests to no longer expect <Primary> to roundtrip
through the accelerator parsing and formatting code.

keymap: Remove virtual modifier mapping functions

These are not used anymore.

Drop the vfuncs and their implementations in the
broadway, x11, wayland and win32 backends as well.

gdk: Make GdkKeymap a private api

We have replacement apis in GdkDevice and GdkDisplay.

imcontextsimple: Disable some win32-only code

This code needs to be redone differently, since keymaps are no
longer going to be exposed. There should really not be this much
ifdef-ed backend-specific code here anyway. Or any, really.

gdk: Clean up GdkModifierType

Remove MOD2..MOD5. Backends are expected to just set
the named modifiers.

gdk: Redo key events

Add all of the keyboard translation results in the key event,
so we can translate the keyboard state at the time the event
is created, and avoid doing state translation at match time.

We actually need to carry two sets of translation results,
since we ignore CapsLock when matching accelerators, in
gdk_event_matches().

At the same time, drop the scancode field - it is only ever
set on win32, and is basically unused in GTK.

Update all callers.

gtk: Remove handling of virtual modifiers

These are going away. We expect to just have
named modifiers in modifier masks now, so we
longer juggle 'real' and 'virtual' modifiers.

events: Stop doing elaborate virtual modifier handling

These are going away in the GdkModifierType cleanup.
Just compare the modifiers we got.

broadway: Stop using MOD2..MOD5

These are going away in the GdkModifierType cleanup.

win32: Stop using MODx modifiers

The win32 backend is using GDK_MOD2_MASK for AltGr,
so define GDK_MOD2_MASK locally to keep this working,
but remove any mention of GDK_MOD3_MASK,...,GDK_MOD5_MASK.

wayland: Stop setting MOD2..MOD5

These are going away in the GdkModifierType cleanup.

gdk: Remove reserved bit from GdkModifierType

This really has no purpose and obscures the header.

gtk: Stop using GDK_MODx_MASK

These are never used in practice, and we never want to
see them in the UI, so stop supporting them. This is
in preparation for cleaning up GdkModifierType.

Rename GDK_MOD1_MASK to GDK_ALT_MASK

We've hardcoded Mod1 = Alt for a long time, there is
no need to keep the confusing naming around anymore.

tests: Stop using gtk_accelerator_set_default_mod_mask

This is not doing anything useful here.

testsuite: Stop using keymap apis

gtk: Use display apis for keymap mapping

This removes the last uses of GdkKeymap from GTK.

gtk: Stop using keymap apis

Stop using most keymap apis in GTK. Only a few calls
related to virtual modifiers are left.

x11: Notify new device properties

wayland: Notify new device properties

Move keymap mapping functions to GdkDisplay

We want to stop exposing GdkKeymap, so this
functionality needs a new home.

Move GdkKeymapKey to gdktypes.h

We are going to use this type in a different header.

gdk: Move modifier mask api to GdkDisplay

GdkKeymap is on the way out.

gdk: Add keymap properties to GdkDevice

GdkKeymap is on the way out.

passwordentry: Show caps lock warning more

Show the caps lock warning regardless of whether we
have the peek icon or not.

Drop some no longer used code

cell renderer accel: Use gdk_event_get_match

Add gdk_event_get_match

This is a counterpart to gdk_event_matches() that can
be used to obtain a shortcut matching an event.

Add gdk_event_matches

Move the elaborate key event matching code from
GtkShortcutTrigger to GdkEvent, which greatly reduces
the amount of keymap api use outside of GDK.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1630

gtk-demo: Fix a crash

Using the assistant demo repeatedly was crashing.

Merge branch 'matthiasc/for-master' into 'master'

shortcuts: documentation fixes

See merge request GNOME/gtk!1628

Merge branch 'matthiasc/for-master' into 'master'

shortcuts: documentation fixes

See merge request GNOME/gtk!1628

shortcuts: documentation fixes

Update symbol lists, and add examples for parsing triggers
and actions.

Update POTFILES.skip

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1626

colorbutton: Don't add a customize menu

The color swatch in the button does not need a
context menu. Turn it off.

colorbutton: Fix focus

The colorbutton contains a button which contains a colorswatch.
We want the focus to go straight to the button, nowhere else,
so mark the swatch as !can-focus.

Adapt tests to match.

demos: Remove can-focus from ui files

Merge branch 'scrollbar-a11y' into 'master'

Add GtkScrollbarAccessible

See merge request GNOME/gtk!1623

scrollbar: Be more defensive

Somehow the scrollbar accessible manages to call
gtk_scrollbar_get_adjustment when priv->range is NULL.
Survive taht.

Merge branch 'shortcut-action-parse' into 'master'

Shortcut action parse

See merge request GNOME/gtk!1625

testsuite: Add tests for parsing shortcut actions

Add an api to parse shortcut actions

This is so we can test this code. While doing it
I also fixed the missing else that was caused it
not to work.

Add GtkScrollbarAccessible

When GtkScrollbar stopped being a GtkRange,
GtkRangeAccessible stopped working for scrollbars.

Update a11y tests to match.

Merge branch 'wip/carlosg/fix-surrounding-text-master' into 'master'

imwayland: Clamp the surrounding string end correctly

Closes #2565

See merge request GNOME/gtk!1621

Merge branch 'focus-test' into 'master'

Focus test

See merge request GNOME/gtk!1622

imwayland: Clamp the surrounding string end correctly

In the paths where len > MAX_LEN and cursor/anchor are separated by
at least MAX_LEN from text edges, we were clamping the right end of
the surrounding string at MAX_LEN. Oops.

This end anchor may go as far as the string length, although just
up to len - MAX_LEN in real terms (due to the condition above that
caches cursor/anchor positions being near enough the text end).

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2565
(cherry-picked from commit d7fb15c822eabe22ffda19892153d69b0f4fc2f3)

testsuite: Add a big focus chain testcase

testsuite: Try harder to record a full focus chain

It is hard to avoid widgets with the same name in a
large ui file - try harder to record a full focus chain
before decide that we've wrapped, by including the widget
address in the comparison. Note that we don't include
the addresses in the generated output, since that would
make expected output vary from run to run.

Merge branch 'gir-fixes' into 'master'

Annotation fixes

See merge request GNOME/gtk!1616

Merge branch 'move-gdkgrab-decl' into 'master'

gdk: Move GdkGrabStatus and GdkGrabOwnership decl.

See merge request GNOME/gtk!1615

Annotation fixes

Match the arguments name in the declaration and the definition, and make
sure we have the proper transfer annotations.

gdk: Move GdkGrabStatus and GdkGrabOwnership decl.

With the removal of `gdk_seat_grab` we do not need to keep the
definitions of `GdkGrabStatus` and `GdkGrabOwnership` public.

Move those definitions to become internal only.

shortcutcontroller: fix transfer full annotation

window: Don't leak toplevel layout

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1611

wayland: Add a comment about keyboard state handling

Clarify a point that took me a few hours to fully track
down, so lets preserve what I found for the next poor
sould coming this way.

a11y tests: Update for focus state changes

a11y: Set focus-related states properly

Entries and menubuttons are no longer focusable themselves,
they have focusable children. Since we don't have accessible
objects for those, transfer the focus-related state (focusable
and focused) from the children to the main accessible object.

window: tie focus to surface state changes

We don't get a focus-out on the event controller, when
the surface is losing keyboard focus, since we are not
moving our focus to some other widget, so we are never
unsetting the mnemonics-visible property. Do that in
response to surface state changes instead.

Merge branch 'cherry-picked-wayland-keyboard-fix' into 'master'

gdk/wayland: Clear modifiers when we lose keyboard focus

Closes #2112

See merge request GNOME/gtk!1610

gdk/wayland: Clear modifiers when we lose keyboard focus

When we `Alt+Tab` away from a GTK application, it loses keyboard focus.
If we don't clear the modifiers, events from other devices that we
receive while unfocused will assume `Alt` is still pressed. This results
in e.g. Firefox navigating through the history instead of scrolling the
page when using the mouse wheel on it.

We don't get any information about modifiers while we are missing
keyboard focus, so assuming no modifiers are active is the best we can
do.

The shell sends us a modifier update immediately before we regain
keyboard focus, so the state shouldn't get out of sync.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/2112

Merge branch 'wip/smcv/libdir-independent-valgrind-suppressions' into 'master'

supp: Use a single suppressions file for lib, lib64 and multiarch

See merge request GNOME/gtk!1603

Merge branch 'im-context-wayland-fixes' into 'master'

Im context wayland fixes

See merge request GNOME/gtk!1605

imcontexsimple: Move Compose file loading to class_init

This ensures that it is done only once.

imcontextsimple: Simpify Compose file loading

We are not loading the Compose file for individual contexts,
it just gets added to a global list. So don't pass an im context
along. This will let us move the loading out of the initialization
of individual contexts, and only do it once.

imcontextsimple: Load ~/.Compose unconditionally

There is nothing display-specific in the ~/.Compose file,
so we can just try to load it without looking at the
display we are on.

imcontextsimple: Add a profiler mark for IO

We are loading a file here, thankfully in a thread, but
we do it every time an entry is created. Add a profiler
mark, to make this visible.

supp: Use a single suppressions file for lib, lib64 and multiarch

In addition to the traditional library directory lib and the 64-bit
multilib directory lib64, this will cover Debian-style multiarch
(lib/x86_64-linux-gnu etc.), Arch Linux 32-bit (lib32), x32 and
various others.

Signed-off-by: Simon McVittie <smcv@debian.org>

Merge branch 'fix-menu-flicker' into 'master'

Fix crossing event generation

Closes #2536

See merge request GNOME/gtk!1601

Fix crossing event generation

We were not properly setting the new_descendent field
in Crossing structs for GTK_CROSSING_OUT events. This
was causing extraneous ::leave signals to be emitted,
and make model buttons in popover menus flicker when
hovered.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2536

Merge branch 'drop-popover-demo' into 'master'

gtk-demo: Remove the popover demo

Closes #2429

See merge request GNOME/gtk!1600

gtk-demo: Remove the popover demo

It doesn't work anymore, since popovers now need
support in the parent, and we don't really need
a demo just for popovers. They are used everywhere
already.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2429

Merge branch 'flow-box-typo' into 'master'

gtkflowbox: Fix a minor typo in the documentation

See merge request GNOME/gtk!1599

gtkflowbox: Fix a minor typo in the documentation

Signed-off-by: Philip Withnall <withnall@endlessm.com>

Merge branch 'matthiasc/for-master' into 'master'

keynav: Fix Shift-Tab

See merge request GNOME/gtk!1598

Merge branch 'cursor-aspect-ratio' into 'master'

Add a cursor-aspect-ratio setting

See merge request GNOME/gtk!1597

Update the settings test

keynav: Fix Shift-Tab

This was broken by the shortcuts branch merge.

Add a cursor-aspect-ratio setting

Add a setting that controls how thick the text caret is drawn.
The default value is 0.04.

3.98.2

Merge branch 'wip/carlosg/stuck-pointer-focus' into 'master'

gtkmain: Do not try to coalesce/ignore crossing events

See merge request GNOME/gtk!1594

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1593

gtkmain: Do not try to coalesce/ignore crossing events

Under grabbing circumstances we used to get several crossing events,
some corresponding to the grab itself and some corresponding to
pointer motion.

The backends now do a better job at keeping those simple, which
means we sit listening for events that don't actually arrive. This
triggers pointer focus issues when dragging windows or opening
grabbing popups.

Actually obey those events, they will be the only ones we get now.

testsuite: Add a test for transforms in builder files

testsuite: Add a test for parsing layout properties

and drop the child properties test that doesn't
do anything anymore.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1591

window: Fix residual resizing issues

After the GdkToplevel split, GtkWindow occasionally
forgets its minimum size. Fix this by keeping a GdkToplevelLayout
around and reusing it.

meson.build: Fix Visual Studio builds

We need to make the C++ language available before we can access the C++
compiler object...

NEWS: Updates

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1589

shortcutcontroller: Document buildable support

testsuite: Add a builder test for shortcut controllers

shortcutcontroller: Fix builder support

The GtkBuilder support was not working anymore after
triggers and actions have been turned into objects.

Fix this.

Add new api to the docs

Merge branch 'wip/hadess/portal-fixes' into 'master'

Portal fixes

See merge request GNOME/gtk!1585

Merge branch 'issue982' into 'master'

gdk: Add gdk_toplevel_inhibit_system_shortcuts API

Closes #982

See merge request GNOME/gtk!1542

filechoosernativeportal: Fall back if portal is too old

Add portal version checking as originally implemented for the GTK 3.x
branch.

See: 35fec1c6b298e0e294530e1ffc8615b5bd261531

Add portal version checking helper

Add gtk_get_portal_interface_version() to check the version of a portal.